Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
funcsom
left a comment
funcsom
left a comment
There was a problem hiding this comment.
우선 정말 홈화면 완성까지 제가 너무 질질끌어서 늦어진 점 죄송합니다ㅜㅜㅜㅜ 그리고 정말 고생 많으셨습니다.. 기존의 코드를 이해하고 수정한다는게 정말 쉽지 않은건데도 불구하고 너무 잘해주셨네요...! 정말 체고예욤....
추가로 제가 피그마에서 언급하는 것을 까먹은 점들이 몇가지 있는데..., 홈화면 곰돌이가 조금 커졌습니다! 반영해주시면 더 풍성한 홈화면이 될 수 있을 것 같아요 ㅎㅎ 그리고 달력 컴포넌트 위치가 header 바로 밑에 붙었답니다!! 그러면 아래의 일기조회 컴포넌트가 더 잘 보일 것 같아요!
항상 고생하십니다 감사합니다..!
감사합니다 ! 작업 속도에 관해서는 전혀....죄송해하지 않아도 됩니다 ㅠㅠㅠ 다들 바쁘게 살고 있어서 ㅎㅎ... 부담 없이 진행했음 합니다 ! 허니베어와 달력 컴포넌트 스타일은 수정해서 반영하겠습니다 ! 감사합니다 ) |
| //라우트 설정 완료되면 주석 해젷 사용하심 될 것 같습니다 ! | ||
| //navigate('/search'); |
| const getSegment = () => { | ||
| const location = useLocation(); | ||
| const path = location.pathname; | ||
| const segments = path.split('/').filter(Boolean); | ||
| const firstSegment = segments[0]; | ||
|
|
||
| return firstSegment; | ||
| } | ||
|
|
||
| const segment = getSegment(); |
There was a problem hiding this comment.
이 state를 useState로 하지 않은 이유가 있을까요 ~??
There was a problem hiding this comment.
MenuHeader 라는 하위 컴포넌트가 Main 컴포넌트와 CalendarPage 컴포넌트라는 완전히 다른 부모 컴포넌트를 가지며 공통된 부모 컴포넌트가 없기 때문에 state 를 하나의 변수로 관리하는 것이 어렵다고 판단하였습니다 !
| const handleSearch = () => { | ||
| alert('삐용'); | ||
| //라우트 설정 완료되면 주석 해젷 사용하심 될 것 같습니다 ! | ||
| //navigate('/search'); | ||
| }; | ||
| const handleCalendar = () => { | ||
| navigate('/calendar'); | ||
| }; | ||
| const handleMyPage = () => { | ||
| navigate('/myPage'); | ||
| }; | ||
| const handleMain = () => { | ||
| navigate('/main'); | ||
| }; |
There was a problem hiding this comment.
반복되는 코드들과 함수들은 이렇게 따로 빼는것이 아니라, 직접 작성하거나, list 형식으로 만들면 좋을 거 같아요 !!
| export const WeatherPlaylistWrapper = styled.div` | ||
| width: 100%; | ||
| margin-top: 19rem; | ||
| z-index: 100; |
There was a problem hiding this comment.
main 에서 이 부분에 z-index를 써준 이유는 뒤에 곰돌이 때문이려나요 ~??
There was a problem hiding this comment.
맞습니다 ㅠㅠㅠ 곰돌이...
그런데 오늘 회의 결과 날씨 부분은 빼기로 하여서, 아마 삭제할 것 같습니다.
| background: #C7C7C7; | ||
| ` | ||
| display: flex; | ||
| width: 32.0rem; |
| export const CalendarIcon = styled(IcCalendarIcon)` | ||
| `; No newline at end of file |
There was a problem hiding this comment.
이제 와서 보니 Ic 태그들을 이렇게 속성이 없어도 스타일로 선언하는게 맞나.. 생각이 드네요. 해당 내용 공부하고 내용 공유 한번 해드릴게요 !! 같이 이야기 해봐요
There was a problem hiding this comment.
맞아요,,, css 속성 적용도 잘 안 되던데 이렇게 하는 게 맞는 방법인가? 생각하면서도 그냥 무지성으로 해버리는....
코드를 작성하는 경험도 다양하게 해보고 싶은데, 저렇게 부수적으로 공부해야될 부분들도 참 많은 것 같네요 ㅠㅠ
저도 해당 부분 관련 내용 한번 찾아보겠습니다 !!
3 participants
Related Issue 🍫
Summary 🍪
Before i request PR review 🍰
0905.mp4